Skip to content

Conversation

@kazuma-desu
Copy link
Contributor

Summary

Readme has some issues that may make it difficult for some users to get started.

Fixed mv command renaming uwu-cli to uwu after moving.

Fixed invalid model name in gemini config in Readme and sample config.

@samheutmaker
Copy link
Contributor

I don't think we want to rename the binary -- we need to reserve the uwu name for the zsh function. The gemini name change looks fine.

```bash
chmod +x dist/uwu-cli
mv dist/uwu-cli /usr/local/bin/uwu-cli
mv dist/uwu-cli /usr/local/bin/uwu

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The uwu-cli binary name is intended and preffered. The binary is not supposed to be called directly -- you are instead supposed to create a function named uwu that calls uwu-cli under the hood. Having the binary called uwu will conflict with the function name.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants